Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Batch Sending of Webhooks #61

Merged
merged 4 commits into from
Feb 16, 2024
Merged

Batch Sending of Webhooks #61

merged 4 commits into from
Feb 16, 2024

Conversation

tsutomi
Copy link
Member

@tsutomi tsutomi commented Feb 16, 2024

Adding the support to send webhooks on batches of events:

  • Definition of the EventNotification type that aggregates multiple events of the same type
  • Webhook factories contract accepting EventNotification to build arrays of webhooks
  • Webhook factory strategy to generate one webhook for multiple events, or one webhook per event in the notification

@tsutomi tsutomi added enhancement New feature or request notifier The notification service labels Feb 16, 2024
@tsutomi tsutomi self-assigned this Feb 16, 2024
@tsutomi tsutomi linked an issue Feb 16, 2024 that may be closed by this pull request
Batch Sending of Webhooks #41
Closed
@tsutomi tsutomi merged commit 3fb75fa into main Feb 16, 2024
1 check passed
@tsutomi tsutomi deleted the 41-batch-sending-of-webhooks branch February 16, 2024 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@tsutomi tsutomi

Labels
enhancement New feature or request notifier The notification service
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Batch Sending of Webhooks
1 participant
@tsutomi